Conversation
|
Hi there, thanks for raising another PR, much appreciated!
Yes, that sounds very reasonable. I don't think we cater to an audience that would ever consider that. Let's remove it.
We could, placing it in DOMAPI would be the in line with the rest.
That is interesting, do you have a stack trace or anything there?
I'm not sure that is worth it. I would add that if a lot of people have this use-case. |
5085d32 to
8e1fe03
Compare
That wasn't a hint, I fully agree that DOMAPI.res is only the place that makes sense based on the structure of the project. ^^ I updated the PR based on your suggestions. |
This draft proposes a very conservative change to improve timeout-related bindings, but is mostly meant to help me align with the project's strategy:
setTimeoutwith a string altogether, as suggested in Improve generated overloads #27?timeoutIdphantom type like in the Core library? If so, should we make use of the existing one or should we declare our own in a place like DOMAPI.res (which makes my editor crash btw)?setTimeout's behaviour with floats isn't specified and thus could be inconsistent across runtimes.